Skip to content

Conversation

@RoseSecurity
Copy link
Contributor

@RoseSecurity RoseSecurity commented Oct 23, 2025

Description

This adds a new domain_name output to expose the domain name attribute from the OpenSearch domain resource, allowing users to reference the domain name in their Terraform configurations for constructing endpoints, configuring DNS records, or referencing the domain in other AWS resources.

Motivation and Context

Breaking Changes

N/A

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

- Changed the default TLS security policy for the OpenSearch domain to
  "Policy-Min-TLS-1-2-PFS-2023-10" for improved security.
- Added a new output "domain_name" to expose the OpenSearch domain name.
@RoseSecurity RoseSecurity changed the title feat(opensearch): update TLS policy and add domain_name output feat(opensearch): update TLS policy and add domain_name output Oct 23, 2025
@RoseSecurity RoseSecurity changed the title feat(opensearch): update TLS policy and add domain_name output Update TLS policy and add domain_name output Oct 23, 2025
Updated the default value for `tls_security_policy` in domain endpoint
options from "Policy-Min-TLS-1-2-2019-07" to
"Policy-Min-TLS-1-2-PFS-2023-10" in variables, wrapper, and documentation.
Also added `domain_name` output to the complete example README for
consistency.
Change the default value for `tls_security_policy` in domain endpoint
options from "Policy-Min-TLS-1-2-PFS-2023-10" to
"Policy-Min-TLS-1-2-2019-07" across documentation, variables, and
module usage. This ensures compatibility with a broader range of
clients and aligns with AWS recommended defaults.
@RoseSecurity RoseSecurity changed the title Update TLS policy and add domain_name output Add domain_name output Oct 23, 2025
@RoseSecurity RoseSecurity changed the title Add domain_name output feat: Add domain_name output Oct 23, 2025
Copy link
Member

@bryantbiggs bryantbiggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@bryantbiggs bryantbiggs merged commit c102fd0 into terraform-aws-modules:master Oct 23, 2025
8 of 11 checks passed
antonbabenko pushed a commit that referenced this pull request Oct 23, 2025
## [2.3.0](v2.2.1...v2.3.0) (2025-10-23)

### Features

* Add `domain_name` output ([#48](#48)) ([c102fd0](c102fd0))
@antonbabenko
Copy link
Member

This PR is included in version 2.3.0 🎉

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add domain_name as output

3 participants